Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Each start key fix #297

Merged
merged 5 commits into from
Sep 4, 2022
Merged

Each start key fix #297

merged 5 commits into from
Sep 4, 2022

Conversation

ziaratban
Copy link
Contributor

Q A
Is bugfix? yes
New feature? no
Breaks BC? no
Tests pass? yes

start index in each() method is 1.
now start from 0.

@rustamwin
Copy link
Member

Duplicate of #291

@rustamwin rustamwin marked this as a duplicate of #291 Feb 11, 2020
@ziaratban
Copy link
Contributor Author

ziaratban commented Feb 11, 2020

Duplicate of #291

Not Duplicate. (see fix each() start key)
better performance for exists() method of Query class exists in master branch.

how i can remove this commit(of master branch) from another branch ?

@samdark
Copy link
Member

samdark commented Feb 14, 2020

@ziaratban by rebasing.

@samdark samdark added this to the 2.1.10 milestone Feb 14, 2020
foreach($cursor as $i => $v)
   exit($i); #return 1   -> bug ...
@samdark samdark modified the milestones: 2.1.10, 2.2.0 Feb 19, 2020
@samdark
Copy link
Member

samdark commented Feb 19, 2020

Could likely cause issues with current code in case it relies on indexes. Should be released in 2.2.

@samdark
Copy link
Member

samdark commented Feb 19, 2020

@ziaratban would you please add a line for CHANGELOG.md?

@samdark samdark added the type:bug Bug label Feb 21, 2020
@samdark samdark merged commit dacb595 into yiisoft:master Sep 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants